-
Notifications
You must be signed in to change notification settings - Fork 730
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve Integration Tests for System Defined IdPs #21883
Improve Integration Tests for System Defined IdPs #21883
Conversation
...s/org/wso2/identity/integration/test/rest/api/server/idp/v1/add-idp-oidc-standard-based.json
Outdated
Show resolved
Hide resolved
.../src/test/java/org/wso2/identity/integration/test/rest/api/server/idp/v1/IdPFailureTest.java
Outdated
Show resolved
Hide resolved
...s/org/wso2/identity/integration/test/rest/api/server/idp/v1/add-idp-oidc-standard-based.json
Show resolved
Hide resolved
80c7626
to
e21d169
Compare
.body("identityProviders.find { it.id == '" + idPId + "' }.name", equalTo(IDP_NAME)) | ||
.body("identityProviders.find { it.id == '" + idPId + "' }.isEnabled", equalTo(true)) | ||
.body("identityProviders.find { it.id == '" + idPId + "' }.self", equalTo(getTenantedRelativePath( | ||
"/api/server/v1/identity-providers/" + idPId, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check whether there is constant for this "/api/server/v1/identity-providers/"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the other similar instances also we have used the same way.
PR builder started |
PR builder completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/12138645129
Proposed changes in this pull request
This PR adds API tests to validate the handling of duplicated scopes in standard-based OIDC IdPs and to validate the search functionality in IdPs.
Related Issue:
Checklist (for reviewing)
General
Functionality
Code
Tests
Security
Documentation